Skip to content

Comments

[IMP]website_sale_stock_ux: only show ribbon if product has no variants#438

Closed
JrAdhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-107325-jr
Closed

[IMP]website_sale_stock_ux: only show ribbon if product has no variants#438
JrAdhoc wants to merge 1 commit intoingadhoc:18.0from
adhoc-dev:18.0-h-107325-jr

Conversation

@JrAdhoc
Copy link
Contributor

@JrAdhoc JrAdhoc commented Jan 5, 2026

No description provided.

Copilot AI review requested due to automatic review settings January 5, 2026 20:06
@roboadhoc
Copy link

Pull request status dashboard

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Este PR mejora el módulo website_sale_stock_ux para mostrar el ribbon y el blur de "Out of stock" solo cuando el producto no tiene variantes configuradas. Esto evita mostrar información engañosa en productos con múltiples variantes donde solo algunas variantes pueden estar sin stock.

  • Se añade la condición len(product.product_variant_ids) == 1 tanto para el ribbon visual como para el efecto blur de imagen
  • Se incrementa la versión del módulo de 18.0.1.1.0 a 18.0.1.2.0 (siguiendo la convención de bump para cambios en vistas)

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
website_sale_stock_ux/views/templates.xml Agrega la verificación de cantidad de variantes a las condiciones del ribbon "Out of stock" y el efecto blur de imagen
website_sale_stock_ux/manifest.py Incrementa la versión del módulo (minor bump) conforme a las mejoras realizadas en las vistas

@cem-adhoc
Copy link
Contributor

@roboadhoc r+

roboadhoc pushed a commit that referenced this pull request Jan 5, 2026
closes #438

Signed-off-by: Manuela Cepeda <cem@adhoc.com.ar>
@roboadhoc roboadhoc closed this Jan 5, 2026
@roboadhoc roboadhoc deleted the 18.0-h-107325-jr branch January 5, 2026 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants